Skip to content

refactor(frontend): decompose useSpecializedAnalysis (#2372)#2456

Merged
mrveiss merged 1 commit intoDev_new_guifrom
refactor/decompose-specialized-analysis
Mar 26, 2026
Merged

refactor(frontend): decompose useSpecializedAnalysis (#2372)#2456
mrveiss merged 1 commit intoDev_new_guifrom
refactor/decompose-specialized-analysis

Conversation

@mrveiss
Copy link
Owner

@mrveiss mrveiss commented Mar 26, 2026

Summary

  • Decomposed useSpecializedAnalysis.ts (583 lines, 5 concerns) into focused composables:
    • useApiEndpointAnalysis.ts (129 lines) — API endpoint coverage scanning
    • useConfigDuplicates.ts (53 lines) — config duplicate detection
    • useEnvironmentAnalysis.ts (120 lines) — env scanning with AI filtering
    • useOwnershipAnalysis.ts (95 lines) — code ownership mapping
    • useCrossLanguageAnalysis.ts (270 lines) — cross-language consistency
  • Original file preserved as 42-line facade with spread returns for full backward compatibility
  • Only consumer (useCodeIntelAnalysis.ts) imports useSpecializedAnalysis and spreads its return — zero breaking changes

Closes #2372

Test plan

  • Pre-commit hooks pass (no console.*, trailing whitespace, etc.)
  • TypeScript compilation passes (npx vue-tsc --noEmit)
  • Analysis views render correctly with no runtime regressions

…posables (#2372)

Split useSpecializedAnalysis.ts (583 lines, 5 concerns) into:
- useApiEndpointAnalysis.ts (129 lines)
- useConfigDuplicates.ts (53 lines)
- useEnvironmentAnalysis.ts (120 lines)
- useOwnershipAnalysis.ts (95 lines)
- useCrossLanguageAnalysis.ts (270 lines)

Original file preserved as 42-line facade that spreads all sub-composable
returns, maintaining full backward compatibility.
@mrveiss mrveiss merged commit 9b14506 into Dev_new_gui Mar 26, 2026
3 of 4 checks passed
@mrveiss mrveiss deleted the refactor/decompose-specialized-analysis branch March 26, 2026 16:57
@github-actions
Copy link

✅ SSOT Configuration Compliance: Passing

🎉 No hardcoded values detected that have SSOT config equivalents!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant